Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for OAuth and Identity Azure SDK #141

Closed
wants to merge 1 commit into from

Conversation

hacker1db
Copy link

  • Generic Oauth Method to support bringing your own Oauth Token
  • Support for Azure.Identity SDK

I took most of this from another PR that has not already been merged and built upon it.

see #105

This is my first real introduction to GO so if I did something that was different then the "standards" please let me know so I can learn.

Thanks!

- Generic Oauth Method to support bringing your own Oauth Token
- Support for Azure.Identity SDK

I steal this from another PR that has not already been merged and built upon it.

see pr#105
@hacker1db
Copy link
Author

@microsoft-github-policy-service agree

@lsuarez5280
Copy link

What's the timeline for completing this pull? I'm very interested in switching to service principal authorization.

@SimonWahlin
Copy link

I'm also very interested in this feature!
@nechvatalp @besmiralia @hacker1db : Any idea of if and when this can be completed?

@hacker1db
Copy link
Author

I have had this PR open for a while and still no response or feedback, it's out of my hands.

@tmeckel
Copy link
Contributor

tmeckel commented Apr 19, 2024

@lsuarez5280 @hacker1db @SimonWahlin nobody at Microsoft really cares about the AzDO GO API. This repo should be considered dead. Even if it's the base for the AzDO Terraform Provider.

The OAuth Service Principal Authentication in PR https://github.com/microsoft/terraform-provider-azuredevops/pull/747/files has been added into the Terraform Provider instead of adding it to this SDK, what would be the very correct way to do so. Because everyone would benefit from it, not only the Terraform provider.

But you can imagine why the people on the provider decided it to do it isolated in the provider: this repo is dead ☠☠

@hacker1db
Copy link
Author

Can we please mark the project as such before others keep trying to contribute to a dead project? Thank you.

@hacker1db
Copy link
Author

Closing per comment -> #141 (comment)

@hacker1db hacker1db closed this Apr 23, 2024
@tmeckel tmeckel mentioned this pull request Sep 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants